Skip to content

fix: resolve 'too many arguments' CLI error - #61

Merged
bradygaster merged 2 commits into
devfrom
squad/59-fix-cli-error
Feb 15, 2026
Merged

bradygaster merged 2 commits into
devfrom
squad/59-fix-cli-error

Conversation

@bradygaster

Copy link
Copy Markdown
Collaborator

Closes #59

Investigated and fixed the argument parsing error that was displaying 'error: too many arguments' above the textbox in the CLI post-0.4.0.

Root Cause

The \�ersion\ field in the YAML frontmatter of \squad.agent.md\ is not a supported property according to the GitHub Copilot CLI agent manifest specification. When the Copilot CLI parser encounters unsupported properties, it interprets them as command-line arguments, causing the 'error: too many arguments' message.

Changes

  • Removed unsupported \�ersion\ field from YAML frontmatter
  • Moved version tracking to HTML comment (immediately after frontmatter closing ---)
  • *Updated \stampVersion()* in index.js to replace HTML comment and Version line instead of YAML fields
  • *Updated
    eadInstalledVersion()*
    to read from HTML comment (with fallback to old format for backward compatibility)
  • Simplified agent name from \Squad (v0.0.0-source)\ to just \Squad\ in YAML frontmatter

Testing

  • All 45 tests pass
  • Version stamping and reading still works during install/upgrade
  • Backward compatible: can read version from both old and new formats during upgrade

The version is still tracked and displayed to users, but no longer conflicts with the Copilot CLI's YAML parser.

bradygaster and others added 2 commits February 14, 2026 19:30
Remove unsupported 'version' field from YAML frontmatter in squad.agent.md.
The 'version' property is not supported in GitHub Copilot CLI agent manifests
and was causing the Copilot CLI parser to display 'error: too many arguments'
above the textbox.

Changes:
- Moved version tracking from YAML frontmatter to HTML comment
- Updated stampVersion() to replace HTML comment and Version line
- Updated readInstalledVersion() to read from HTML comment (with fallback)
- Removed version from name field (was 'Squad (v0.0.0-source)', now 'Squad')

The version is still tracked and stamped during install/upgrade, but no
longer conflicts with the Copilot CLI's YAML parser.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bradygaster bradygaster added this to the v0.4.1 milestone Feb 15, 2026
@bradygaster
bradygaster merged commit 93485c0 into dev Feb 15, 2026
1 check passed
@bradygaster
bradygaster deleted the squad/59-fix-cli-error branch February 15, 2026 09:35
joniba pushed a commit to joniba/squad that referenced this pull request Mar 26, 2026
Add worktree-based parallel agent isolation:
- SKILL.md: coordinator create/spawn/cleanup flow
- create-worktree.ps1: creates worktree + branch for an issue
- cleanup-worktree.ps1: removes a single worktree after merge
- cleanup-all-worktrees.ps1: prunes all merged/orphaned worktrees
- .gitignore: exclude worktrees/ directory

Resolves github#61. Design approved in github#60.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
joniba pushed a commit to joniba/squad that referenced this pull request Mar 26, 2026
Add worktree-based parallel agent isolation:
- SKILL.md: coordinator create/spawn/cleanup flow
- create-worktree.ps1: creates worktree + branch for an issue
- cleanup-worktree.ps1: removes a single worktree after merge
- cleanup-all-worktrees.ps1: prunes all merged/orphaned worktrees
- .gitignore: exclude worktrees/ directory

Resolves github#61. Design approved in github#60.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant